Skip to content

Bump/gradle 8.5#797

Merged
lanarimarco merged 4 commits intodevelopfrom
bump/gradle-8-5
Mar 25, 2026
Merged

Bump/gradle 8.5#797
lanarimarco merged 4 commits intodevelopfrom
bump/gradle-8-5

Conversation

@lanarimarco
Copy link
Copy Markdown
Collaborator

Description

This pull request primarily updates the Gradle wrapper to version 8.5 and makes several improvements to test robustness and script clarity. The most significant changes are grouped below.

Build Tooling Updates:

  • Upgraded the Gradle wrapper from version 8.1.1 to 8.5 by updating the distributionUrl in gradle-wrapper.properties.
  • Updated metadata and URLs in the gradlew script to reflect current Gradle project structure and removed the SPDX license identifier for consistency. [1] [2]

Script Improvements:

  • Improved the method for determining APP_HOME in the gradlew script for better reliability across environments.
  • Cleaned up error messages in gradlew.bat by removing redirection to stderr and making output more readable. [1] [2]
  • Removed the SPDX license identifier from gradlew.bat for consistency with other scripts.

Test Robustness:

  • Enhanced test assertions in InterpreterTest.kt by normalizing line endings and stripping column positions from error messages, making tests less brittle to formatting changes. [1] [2] [3]

Related to # (issue)

Checklist:

  • If this feature involves RPGLE fixes or improvements, they are well-described in the summary.
  • There are tests for this feature.
  • RPGLE code used for tests is easily understandable and includes comments that clarify the purpose of this feature.
  • The code follows Kotlin conventions (run ./gradlew ktlintCheck).
  • The code passes all tests (run ./gradlew check).
  • Relevant documentation is included in the docs directory.

@lanarimarco lanarimarco changed the title Bump/gradle 8 5 Bump/gradle 8.5 Mar 25, 2026
@lanarimarco lanarimarco merged commit 6c6691d into develop Mar 25, 2026
1 check passed
@lanarimarco lanarimarco deleted the bump/gradle-8-5 branch March 25, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant